Skip to content

fix(playtest): 长动作提示词不再撑变控制栏 - #922

Merged
xyh202131 merged 1 commit into
1024XEngineer:mainfrom
xyh202131:fix/playtest-long-prompt-layout
Aug 28, 2026
Merged

fix(playtest): 长动作提示词不再撑变控制栏#922
xyh202131 merged 1 commit into
1024XEngineer:mainfrom
xyh202131:fix/playtest-long-prompt-layout

Conversation

@xyh202131

Copy link
Copy Markdown
Contributor

Summary

  • constrain the Playtest current-action slot to a stable responsive width
  • truncate prompt-like action names without stretching the control dock
  • preserve the full action name on hover and add regression coverage

Testing

  • npm test (89 files, 1377 tests)
  • npm run typecheck
  • npm run lint
  • npm run build

Keep prompt-like action names from stretching the control dock while preserving the full text on hover.

Co-Authored-By: Codex <codex@openai.com>
@vercel

vercel Bot commented Aug 28, 2026

Copy link
Copy Markdown

Deployment failed for project windup with the following error:

Resource is limited - try again in 24 hours (more than 100, code: "api-deployments-free-per-day").

Learn More: https://vercel.com/huyan-s-projects1?upgradeToPro=build-rate-limit

@github-actions

Copy link
Copy Markdown

⚠️ @xyh202131,此 PR 尚未关联 issue。请在 PR 描述中使用 Closes #123 等关闭关键字;更新描述后,此提醒将自动标记为已解决。

@fennoai fennoai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed the complete fixed diff for the Playtest workbench control dock and its regression coverage. The bounded responsive slot, min-w-0, truncation, and hover title are consistent with the existing flex layout and preserve the current action label without introducing a correctness regression. git diff --check passes. The focused Vitest command could not run in this workspace because the local vitest executable is not runnable (Permission denied).

@codecov

codecov Bot commented Aug 28, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #922      +/-   ##
==========================================
- Coverage   86.19%   86.12%   -0.08%     
==========================================
  Files         306      306              
  Lines       23929    24037     +108     
  Branches     3629     3646      +17     
==========================================
+ Hits        20626    20701      +75     
- Misses       2334     2361      +27     
- Partials      969      975       +6     
Flag Coverage Δ
backend 91.27% <ø> (+0.01%) ⬆️
frontend 79.83% <100.00%> (-0.16%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
frontend/src/pages/playtest/workbench/index.tsx 92.50% <100.00%> (+0.06%) ⬆️

... and 10 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@xyh202131
xyh202131 merged commit 992dadf into 1024XEngineer:main Aug 28, 2026
8 of 9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants